Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] Autorise les lignes du tableaux à avoir une cellule de type heading scope "row" (PIX-16895) #838

Merged
merged 3 commits into from
Mar 7, 2025

Conversation

xav-car
Copy link
Contributor

@xav-car xav-car commented Mar 6, 2025

🎄 Problème

Actuellement la tableau ne permet pas d'avoir sur une ligne une cellule de type heading

🎁 Proposition

Ajouter la possibilité d'avoir une cellule de type heading en la scopant avec scope="row"

🌟 Remarques

RAS

🎅 Pour tester

Mettre le boolean isMainRow à true. et vérifier que nous avons bien un th pour chaque ligne avec le scope row

@xav-car xav-car self-assigned this Mar 6, 2025
@pix-bot-github
Copy link

Une fois l'application déployée, elle sera accessible à cette adresse https://ui-pr838.review.pix.fr
Les variables d'environnement seront accessibles sur scalingo https://dashboard.scalingo.com/apps/osc-fr1/pix-ui-review-pr838/environment

@xav-car xav-car changed the title [TECH] Autorise les lignes du tableaux à avoir une cellule de type heading scope "row" [TECH] Autorise les lignes du tableaux à avoir une cellule de type heading scope "row" (PIX-16895) Mar 6, 2025
@xav-car xav-car force-pushed the enhance-pix-table-a11y branch from a93f1ed to a633108 Compare March 6, 2025 13:24
@xav-car xav-car added 👀 Func Review Needed cross-team Toutes les équipes de dev labels Mar 6, 2025
@xav-car xav-car force-pushed the enhance-pix-table-a11y branch 4 times, most recently from 0a0144c to a139f60 Compare March 6, 2025 16:08
@mcampourcy
Copy link
Contributor

Pour moi tout est OK, mais petite question : à quoi ça sert ? Un heading ne suffit pas ?

@xav-car
Copy link
Contributor Author

xav-car commented Mar 7, 2025

Pour moi tout est OK, mais petite question : à quoi ça sert ? Un heading ne suffit pas ?

attributs scope accede . Ci dessous l'explication de l'attribut scope col / row pour les tableaux.

Site WCAG21 - H63

( c'est une évolution suite à ma formation speed training accessibilité que j'ai eu a Znk . )

Copy link
Contributor

@mcampourcy mcampourcy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tech review OK

@xav-car Merci pour les explications ! accessoirement, j'aurais jamais pensé aux tableaux à doubles entrées (qui nécessitent donc deux headers)

@mcampourcy
Copy link
Contributor

Func review OK
Capture d’écran 2025-03-07 à 09 25 44

@xav-car
Copy link
Contributor Author

xav-car commented Mar 7, 2025

Tech review OK

@xav-car Merci pour les explications ! accessoirement, j'aurais jamais pensé aux tableaux à doubles entrées (qui nécessitent donc deux headers)

Même sans double entrée c'est aussi utile : cf Site WCAG21 - H63

@pix-service-auto-merge pix-service-auto-merge merged commit bc30ba5 into dev Mar 7, 2025
5 of 7 checks passed
@pix-service-auto-merge pix-service-auto-merge deleted the enhance-pix-table-a11y branch March 7, 2025 13:07
pix-service-auto-merge pushed a commit that referenced this pull request Mar 7, 2025
## [54.14.1](v54.14.0...v54.14.1) (2025-03-07)

### 🏗️ Tech

- [#838](#838) Autorise les lignes du tableaux à avoir une cellule de type heading scope "row" (PIX-16895)
@pix-service-auto-merge
Copy link
Contributor

🎉 This PR is included in version 54.14.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants